Two-Stage Review

概述

Two-Stage Review 是一种质量保证流程，在 Subagent-Driven Development 中使用两个独立的评审阶段来确保任务完成质量：第一阶段评审规格合规性，第二阶段评审代码质量。这种分离确保了"做正确的事"先于"做得对不对"。

关键内容

1. 第一阶段：规格合规性评审（Spec Compliance Review）：
2. 评审员角色：怀疑论者（Skeptic），不相信实现子 Agent 的自述
3. 检查内容：
   • 规格要求的每一个功能点都实现了吗？
   • 有没有实现规格没有要求的额外功能（违反 YAGNI）？
   • 读的是真实代码，而不是相信实现者说"完成了"

4. 设计理念：No point reviewing code quality if the implementation doesn't match requirements

5. 第二阶段：代码质量评审（Code Quality Review）：

6. 触发条件：仅在 Phase A 通过后才触发

7. 检查内容：

   • 代码是否遵循项目已有的约定和模式？
   • 错误处理是否完整？类型安全？防御性编程？
   • 代码组织、命名规范、可维护性
   • 新创建的文件或修改的文件是否已经过大？（大文件往往意味着职责不清，需要拆分）

8. 评审顺序的重要性：

9. 先确认"做了正确的事"，再审"做得对不对"

10. 顺序不能颠倒，因为如果实现不符合规格，高质量的代码也没有意义

11. 评审流程：

12. 派遣 spec-reviewer 子 Agent 进行规格合规性评审
13. 通过后派遣 code-quality-reviewer 子 Agent 进行代码质量评审

14. 任何阶段发现问题都会回到实现子 Agent 进行修复，然后重新评审

15. 优势：

16. 职责分离：不同评审关注不同方面，避免混淆
17. 质量保证：双重保障确保实现既符合规格又具备高质量
18. 清晰反馈：明确区分规格问题和质量问题，便于修复

来源

• 05-subagent-driven-development — Two-Stage Review 流程的详细说明

相关

• subagent-driven-development Skill — part_of
• Quality Assurance — implements
• Specification Compliance — relates_to
• Code Quality Review — extends
• Multi-Agent Orchestration — relates_to